Skip to content

Use messages in summary printer #3028

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 12 commits into from
Sep 19, 2025
Merged

Use messages in summary printer #3028

merged 12 commits into from
Sep 19, 2025

Conversation

mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Jul 17, 2025 •
edited
Loading

🤔 What's changed?

Switch the summary and progress plugins to messages and extract to the pretty-formatter module.

⚡️ What's your motivation?

Part of #3001

🏷️ What kind of change is this?

  • 🏦 Refactoring/debt/DX (improvement to code design, tooling, etc. without changing behaviour)

♻️ Anything particular you want feedback on?

The summary is mostly a visual report. Does it look good? Can it be better?

📋 Checklist:

@mpkorstanje mpkorstanje force-pushed the message-based-progress-and-rerun branch from 300d2b1 to da22af5 Compare July 17, 2025 14:52
@mpkorstanje mpkorstanje changed the title [Core] Message based progress and rerun formatter Use messages in rerun formatter Jul 17, 2025
@mpkorstanje mpkorstanje force-pushed the message-based-progress-and-rerun branch from da22af5 to 15cfe0f Compare July 17, 2025 15:04
@mpkorstanje mpkorstanje mentioned this pull request Jul 17, 2025
Open
11 tasks
@mpkorstanje mpkorstanje force-pushed the message-based-progress-and-rerun branch from 1e9b7ab to 792503f Compare September 11, 2025 14:46
@mpkorstanje mpkorstanje changed the title Use messages in rerun formatter Use messages in summary formatter Sep 14, 2025
@mpkorstanje mpkorstanje force-pushed the message-based-progress-and-rerun branch from f9490c5 to 152bbf5 Compare September 14, 2025 16:54
@mpkorstanje mpkorstanje marked this pull request as ready for review September 19, 2025 01:55
@mpkorstanje mpkorstanje merged commit 63ad555 into main Sep 19, 2025
6 checks passed
@mpkorstanje mpkorstanje deleted the message-based-progress-and-rerun branch September 19, 2025 01:58
@mpkorstanje mpkorstanje changed the title Use messages in summary formatter Use messages in summary printer Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mpkorstanje